Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove some code to reduce the size of the mobile inference library. #5252

Merged
merged 9 commits into from
Nov 14, 2017

Conversation

hedaoyuan
Copy link
Contributor

@hedaoyuan hedaoyuan commented Oct 31, 2017

The code that removed from the mobile inference library:

  1. SparseRowCpuMatrix
  2. SparseMatrix
  3. Remove some useless layers

Copy link
Contributor

@Xreki Xreki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested this PR in iOS app, the static libraries' size are reduced from

-rw-r--r--  1 liuyiqun01  INTERNAL\Domain Users   8663872 11 14 11:31 libpaddle_capi_engine.a
-rw-r--r--  1 liuyiqun01  INTERNAL\Domain Users  14365784 11 14 11:31 libpaddle_capi_layers.a
-rw-r--r--  1 liuyiqun01  INTERNAL\Domain Users  23029560 11 14 11:31 libpaddle_capi_whole.a

to

-rw-r--r--  1 liuyiqun01  INTERNAL\Domain Users   6863352 11 14 17:30 libpaddle_capi_engine.a
-rw-r--r--  1 liuyiqun01  INTERNAL\Domain Users  10040944 11 14 17:30 libpaddle_capi_layers.a
-rw-r--r--  1 liuyiqun01  INTERNAL\Domain Users  16904200 11 14 17:30 libpaddle_capi_whole.a

@hedaoyuan hedaoyuan merged commit 562599e into PaddlePaddle:develop Nov 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants